Skip to content

Adds support for render detect for Android 6+#2027

Merged
bidetofevil merged 2 commits intomainfrom
hho/add-legacy-render
Mar 13, 2025
Merged

Adds support for render detect for Android 6+#2027
bidetofevil merged 2 commits intomainfrom
hho/add-legacy-render

Conversation

@bidetofevil
Copy link
Copy Markdown
Contributor

@bidetofevil bidetofevil commented Mar 12, 2025

Goal

Use an alternate way to detect that a frame was been delivered that works for Android 12 and Android 6 to 9. Implement a new DrawEventEmitter that uses this alternative method, and supply when the other one doesn't work.

All the unit tests have been updated to reflect this new reality, but the integration tests don't check for hardware based render events. But with this, we may be able to test this by simulating this, so stay tuned!

Testing

Manually tested for a subset of real devices/apps. Will test it more comprehensively.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 12, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@bidetofevil bidetofevil force-pushed the hho/add-legacy-render branch from c0ca9bf to fae07ea Compare March 12, 2025 22:40
@bidetofevil bidetofevil marked this pull request as ready for review March 12, 2025 22:48
@bidetofevil bidetofevil requested a review from a team as a code owner March 12, 2025 22:48
@bidetofevil bidetofevil force-pushed the hho/add-legacy-render branch from fae07ea to a22e3a9 Compare March 13, 2025 05:16
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 13, 2025

Codecov Report

Attention: Patch coverage is 68.75000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 85.37%. Comparing base (4c8903d) to head (4ba108f).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...dk/internal/capture/activity/UiLoadTraceEmitter.kt 33.33% 0 Missing and 4 partials ⚠️
...android/embracesdk/internal/ui/DrawEventEmitter.kt 62.50% 1 Missing and 2 partials ⚠️
...cesdk/internal/handler/AndroidMainThreadHandler.kt 50.00% 1 Missing and 1 partial ⚠️
...bracesdk/internal/ui/HandlerMessageDrawDetector.kt 87.50% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2027      +/-   ##
==========================================
+ Coverage   85.23%   85.37%   +0.13%     
==========================================
  Files         464      466       +2     
  Lines       10512    10523      +11     
  Branches     1556     1558       +2     
==========================================
+ Hits         8960     8984      +24     
+ Misses        875      858      -17     
- Partials      677      681       +4     
Files with missing lines Coverage Δ
.../embracesdk/internal/capture/activity/UiLoadExt.kt 98.85% <ø> (+6.80%) ⬆️
...internal/capture/startup/AppStartupTraceEmitter.kt 85.49% <100.00%> (-0.08%) ⬇️
...internal/injection/DataCaptureServiceModuleImpl.kt 95.65% <100.00%> (+3.65%) ⬆️
...bracesdk/internal/ui/HandlerMessageDrawDetector.kt 87.50% <87.50%> (ø)
...cesdk/internal/handler/AndroidMainThreadHandler.kt 60.00% <50.00%> (-15.00%) ⬇️
...android/embracesdk/internal/ui/DrawEventEmitter.kt 62.50% <62.50%> (ø)
...dk/internal/capture/activity/UiLoadTraceEmitter.kt 87.26% <33.33%> (+6.36%) ⬆️
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@fractalwrench fractalwrench left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor Author

bidetofevil commented Mar 13, 2025

Merge activity

  • Mar 13, 10:35 AM EDT: A user started a stack merge that includes this pull request via Graphite.
  • Mar 13, 10:35 AM EDT: Graphite rebased this pull request as part of a merge.
  • Mar 13, 10:53 AM EDT: A user merged this pull request with Graphite.

@bidetofevil bidetofevil force-pushed the hho/add-legacy-render branch from a22e3a9 to 4ba108f Compare March 13, 2025 14:35
@bidetofevil bidetofevil merged commit e1aac92 into main Mar 13, 2025
11 checks passed
@bidetofevil bidetofevil deleted the hho/add-legacy-render branch March 13, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants